Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean up inactive working groups #693

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thesuperzapper
Copy link
Member

This PR cleans up our "Working Group" and "Special Interest Group" definitions to align with the current state of the project:

  1. Marks the following groups as inactive:
    • sig-feature-store
    • sig-onprem
    • wg-deployment
    • wg-serving
  2. Removes incorrect meeting definitions:
    • (only the AutoML ones were kept, because the rest were wrong)
    • (the source of truth is really the main community calendar anyway, so its probably not necessary to list them again here)
  3. Updates the "leads" of some groups:
  4. Updates the "subprojects" of some groups:
    • wg-notebooks:
      • Replaced the many links with a single link to the main "components" folder of the kubeflow/kubeflow repo.
    • wg-training:
      • Replaced the archived components with the unified kubeflow/training-operator and still independent kubeflow/mpi-operator

@thesuperzapper
Copy link
Member Author

@jbottum @james-jwu this is a community-level cleanup, so I would appreciate your review of this update so we present correct information about the working groups to users.

/assign @jbottum @james-jwu
/cc @kimwnasptd

Copy link
Member

@andreyvelich andreyvelich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to review from WG owners on this PR.
/assign @kubeflow/wg-training-leads @kubeflow/wg-pipeline-leads @kubeflow/wg-notebooks-leads @kubeflow/wg-manifests-leads

@@ -378,6 +291,7 @@ workinggroups:
- https://raw.githubusercontent.com/kubeflow/pipelines/master/OWNERS
- dir: wg-serving
name: Serving
inactive: true
Copy link
Member

@andreyvelich andreyvelich Jan 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we marking Serving WG as inactive ?
The community is still active even that KServe locates in a separate GitHub org right now.
I guess, we didn't make a final decision what we should do with the Serving part of ML lifecycle in Kubeflow.
cc @yuzisun @terrytangyuan @sivanantha321

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

People are still joining the kserve WG meeting from the kubeflow calendar.

owners:
- https://raw.githubusercontent.com/kubeflow/kubeflow/master/components/tensorflow-notebook-image/OWNERS
- name: notebooks-controller
- name: kubeflow-components
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we name it as Kubeflow Components ?

time: "15:00"
tz: ET (Eastern Time)
frequency: biweekly
url: https://us02web.zoom.us/j/87159010361?pwd=c09wTXk2dmJPNmpkTDFTb0o0cnY2Zz09
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we are going to remove meetings from the wgs.yaml, we need to add link in each WG README to the Kubeflow Website: https://www.kubeflow.org/docs/about/community/#kubeflow-community-calendars

- https://raw.githubusercontent.com/kubeflow/tf-operator/master/OWNERS
- name: xgboost-operator
- https://github.com/kubeflow/mpi-operator/blob/master/OWNERS
- name: training-operator
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would post-pone this change until we archive all other Distributed Training repos (e.g. Fate Operator: https://github.com/kubeflow/fate-operator)

- github: yanniszark
name: Yannis Zarkadas
company: Arrikto
company: Canonical
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kimwnasptd @StefanoFioravanzo Isn't @juliusvonkohout part of Manifests WG ?

Copy link
Member

@juliusvonkohout juliusvonkohout Jan 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, please add me like Kimonas as seen in the google ACLs. company: DHL

wg-notebooks-leads:
- StefanoFioravanzo
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@StefanoFioravanzo is active again within Kueflow

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@juliusvonkohout thanks for mentioning this. I won't be contributing code anytime soon I think. In quality of release product manager I am contributing to the notebooks wg only in terms of reviewing design docs, helping with the roadmap and aligning with the user feedback we gather.

@@ -17,17 +17,11 @@ aliases:
- vpavlin
- yanniszark
wg-manifests-leads:
Copy link
Member

@juliusvonkohout juliusvonkohout Jan 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am missing there, even though i am in the google acls as wg-manifests leads. Please add me.

tech_leads:
- github: StefanoFioravanzo
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@StefanoFioravanzo seems to be active again

|[Pipelines](wg-pipelines/README.md)|area/wg-pipelines|* [James Wu](https://github.com/james-jwu), Google<br>* [Karthik Ramachandran](https://github.com/kramachandran), Google<br>* [Pavel Dournov](https://github.com/paveldournov), Google<br>|* [Slack](https://kubeflow.slack.com/messages/)<br>* [Mailing List](https://groups.google.com/forum/#!forum/kubeflow-discuss)|
|[Serving](wg-serving/README.md)|area/wg-serving|* [Animesh Singh](https://github.com/animeshsingh), IBM<br>* [Clive Cox](https://github.com/cliveseldon), Seldon<br>* [Ellis Tarn](https://github.com/ellistarn), AWS<br>* [Rakesh Kalekar](https://github.com/rakelkar), Microsoft<br>* [Dan Sun](https://github.com/yuzisun), Bloomberg<br>|* [Slack](https://kubeflow.slack.com/messages/kfserving)<br>* [Mailing List](https://groups.google.com/forum/#!forum/kubeflow-discuss)|* Regular WG Meeting: [Wednesdays at 09:00 PT (Pacific Time) (biweekly)]()<br>
|[Training](wg-training/README.md)|area/wg-training|* [Ce Gao](https://github.com/gaocegege), Caicloud<br>* [Johnu George](https://github.com/johnugeorge), Nutanix<br>* [Yuan Tang](https://github.com/terrytangyuan), Red Hat<br>|* [Slack](https://kubeflow.slack.com/messages/wg-training)<br>* [Mailing List](https://groups.google.com/forum/#!forum/kubeflow-discuss)|* Regular WG Meeting (Asia & Europe friendly): [Wednesdays at 03:00 PT (Pacific Time) (monthly - first Wednesday every month)]()<br>* Regular WG Meeting (US friendly): [Wednesdays at 10:00 PT (Pacific Time) (monthly - third Wednesday every month)]()<br>
|[Manifests](wg-manifests/README.md)|area/wg-manifests|* [Kimonas Sotirchos](https://github.com/kimwnasptd), Canonical<br>|* [Slack](https://kubeflow.slack.com/messages/wg-manifests)<br>* [Mailing List](https://groups.google.com/forum/#!forum/kubeflow-discuss)|
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am missing here as well Julius von Kohout, DHL

* Yao Xiao (**[@PatrickXYS](https://github.com/PatrickXYS)**), AWS
* Kimonas Sotirchos (**[@kimwnasptd](https://github.com/kimwnasptd)**), Arrikto
* Yannis Zarkadas (**[@yanniszark](https://github.com/yanniszark)**), Arrikto
* Kimonas Sotirchos (**[@kimwnasptd](https://github.com/kimwnasptd)**), Canonical
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am missing here as well Julius von Kohout, DHL

company: Arrikto
- github: elikatsis
name: Ilias Katsakioris
company: Arrikto
- github: kimwnasptd
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and here as well

company: Arrikto
- github: elikatsis
name: Ilias Katsakioris
company: Arrikto
- github: kimwnasptd
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and here as well

Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@terrytangyuan
Copy link
Member

terrytangyuan commented Jan 25, 2024

We also need evidence for the measure of inactivitiy, e.g. using devstats.

@bhack
Copy link

bhack commented Jan 30, 2024

We also need evidence for the measure of inactivitiy, e.g. using devstats.

Is there anyone finalizing this cncf/sandbox#196

@thesuperzapper
Copy link
Member Author

@terrytangyuan this PR marks the following SIGs / WGs as inactive:

  • sig-feature-store
  • sig-onprem
  • wg-deployment
  • wg-serving

None of these groups have met for a number of years (officially on the calendar), and the WGs components are either archived (kfctl for wg-deployment), or have been moved to a new org (kserve for wg-serving).

Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pending KSC discussion

/hold

Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: thesuperzapper
Once this PR has been reviewed and has the lgtm label, please ask for approval from james-jwu. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants